-
Notifications
You must be signed in to change notification settings - Fork 61
{2023.06}[2023b] Ginkgo 1.9.0 #1106
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Instance
|
The update is good, but please target it to https://github.com/EESSI/software-layer/blob/2023.06-software.eessi.io/easystacks/software.eessi.io/2023.06/eessi-2023.06-eb-5.1.0-2023b.yml |
bot: build repo:eessi.io-2023.06-software instance:eessi-bot-mc-aws arch:aarch64/neoverse_v1 |
Updates by the bot instance
|
New job on instance
|
@pratikvn The build is failing in the configure step due to being in a Slurm job context and not having enough slots. I'll allow oversubscription to push the MPI tests through. In case you want to crosscheck, the general configuration looked like:
|
easystacks/software.eessi.io/2023.06/eessi-2023.06-eb-5.1.0-2023b.yml
Outdated
Show resolved
Hide resolved
bot: build repo:eessi.io-2023.06-software instance:eessi-bot-mc-aws arch:aarch64/neoverse_v1 |
New job on instance
|
@pratikvn Ginkgo is a little too clever and is querying the number of available slots and filtering tests as a result. This is leading to CMake errors:
Can I force it to not do the filtering? |
I think I have found a workaround by adding |
@pratikvn I managed to get Ginkgo to compile for Neoverse_V1 but I am seeing a single failing test:
How concerned should I be about that? If I shouldn't be, how do I skip that test? |
Hi @ocaisa , thank you for looking into this.
|
@pratikvn We did a reorganisation of this repo last night, so I am going to replace this PR so that we can proceed. I got a working recipe (at least for Neoverse V2) at easybuilders/easybuild-easyconfigs#23078 |
easystacks/software.eessi.io/2023.06/eessi-2023.06-eb-5.1.0-2023b.yml
Outdated
Show resolved
Hide resolved
Replacing this by #1127 |
Just changing the target branch to |
This PR adds Ginkgo to the easystack. Please feel free to give feedback.